Skip to content

Integrating maven#2

Merged
arcuri82 merged 5 commits intomasterfrom
integrating-maven
May 19, 2025
Merged

Integrating maven#2
arcuri82 merged 5 commits intomasterfrom
integrating-maven

Conversation

@omursahin
Copy link
Copy Markdown
Contributor

No description provided.

omursahin added 4 commits May 14, 2025 13:45
Now, it can be build with maven,
we can debug using "yarn debug"
Updated readme
@omursahin omursahin requested a review from arcuri82 May 14, 2025 20:02
@@ -0,0 +1,146 @@
import {render, waitFor, screen, act, fireEvent} from '@testing-library/react';
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would refactor the location of this folder. we have src for all files, and src/tests just for this specific test setup? maybe better to have src and src-e2e as siblings, or src/app and src/e2e

Comment thread web-report/README.md
yarn dev
```

The development server will start at `http://localhost:5173` by default.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

any specific reason for such odd 5173 number? why not 8080 (standard in Java)? or 3000 (standard in nodejs)?

@omursahin omursahin requested a review from arcuri82 May 19, 2025 09:44
@arcuri82 arcuri82 merged commit 08f1e20 into master May 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants